Skip to content

Restore both Sessions tray scroll positions#8

Merged
lookfirst merged 1 commit into
mainfrom
feat/persist-session-tray-scroll
Jul 21, 2026
Merged

Restore both Sessions tray scroll positions#8
lookfirst merged 1 commit into
mainfrom
feat/persist-session-tray-scroll

Conversation

@lookfirst

Copy link
Copy Markdown
Collaborator

What changed

  • persists the Sessions tray's right-hand detail scrollbar in addition to the existing session-list scrollbar
  • stores detail scroll positions independently by session id
  • restores the selected session and both navigation positions after a reload
  • documents the behavior and adds preference/component coverage

Why

The Sessions tray has two independent scrolling regions. Only the left session list was persisted, so reloading could restore the selected ride while losing the rider's position within its detailed metrics and charts.

Impact

Riders return to the same session, list position, and detail position after reloading. Switching to another session does not inherit an unrelated ride's detail offset.

Validation

  • bun run ci
  • 218 tests passing
  • Tailwind diagnostics
  • TypeScript type check
  • production Vite build

@lookfirst
lookfirst merged commit 9ffa972 into main Jul 21, 2026
1 check passed
@lookfirst
lookfirst deleted the feat/persist-session-tray-scroll branch July 21, 2026 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant